Skip to content

Sai krishna fix add materials#4493

Merged
one-community merged 7 commits intodevelopmentfrom
SaiKrishna_FixAddMaterials
Feb 21, 2026
Merged

Sai krishna fix add materials#4493
one-community merged 7 commits intodevelopmentfrom
SaiKrishna_FixAddMaterials

Conversation

@saikrishnaraoyadagiri
Copy link
Copy Markdown
Contributor

image

Related PRS (if any):

This frontend PR is related to the development branch backend PR.

Main changes explained:

-Updated addmaterials.jsx page to update the purchase date and phone number validation

  • Updated addMaterials.test.jsx to update the countrycode field
  • Updated package.json - to add libphonenumber-js

How to test:

  1. check into current branch
  2. do npm install and ... to run this PR locally
  3. Clear site data/cache
  4. log as admin user
  5. go to http://localhost:5173/bmdashboard/materials/add
  6. verify Purchase date field and phone number, purchase date field shouldn;t accept future dates and phone number field should accept valid phone numbers only

Screenshots or videos of changes:

date.and.number.mp4

@netlify
Copy link
Copy Markdown

netlify Bot commented Dec 4, 2025

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit a6c6a2d
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/699803f1d81ec7000924fd81
😎 Deploy Preview https://deploy-preview-4493--highestgoodnetwork-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Dec 4, 2025

@one-community one-community added the High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible label Feb 18, 2026
Copy link
Copy Markdown

@Anusha-Gali Anusha-Gali left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi Sai Krishna,

I have reviewed your PR locally and was unable to test it much due to the application logging out continously.

4493.mp4
Image Image Image

@sonarqubecloud
Copy link
Copy Markdown

@saikrishnaraoyadagiri
Copy link
Copy Markdown
Contributor Author

@Anusha-Gali logout issue is fixed, please review

Copy link
Copy Markdown

@Anusha-Gali Anusha-Gali left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi Sai Krishna,

Thanks for fixing the issue. I was able to test the application and the conditions work as per requirement.

Image Image Image

@one-community
Copy link
Copy Markdown
Member

Thank you all, merging!

@one-community one-community merged commit ebb8f14 into development Feb 21, 2026
10 checks passed
pavanputti pushed a commit that referenced this pull request Feb 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants